Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Jul 15, 2025

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

If all target shards, excluding skipped shards, fail, we should fail the
entire query regardless of the partial_results configuration or
skip_unavailable setting. This behavior does not fully align with the
search API, where skip_unavailable ignores all failures from remote
clusters and only fails the request when all shards in the local cluster
fail. However, we believe the proposed behavior is more sensible than
the existing behavior in the search API.

Closes elastic#128994

(cherry picked from commit 8f6f763)

# Conflicts:
#	x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/EsqlNodeFailureIT.java
@dnhatn dnhatn added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jul 15, 2025
@elasticsearchmachine elasticsearchmachine merged commit d217794 into elastic:8.19 Jul 16, 2025
23 checks passed
@dnhatn dnhatn deleted the backport/8.19/pr-131177 branch July 16, 2025 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.19.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants